$mol_number dot minus constraint #717
Conversation
number/number.view.ts
Outdated
| const dot_pos = next.indexOf('.') | ||
|
|
||
| if (dot_pos !== -1) { | ||
| next = next.slice(0, dot_pos) + '.' + next.slice(dot_pos + 1).replace(/\./g, '') |
There was a problem hiding this comment.
А зачем ты пересоздаёшь число? value_normalized возвращает его в нормальной форме.
There was a problem hiding this comment.
value_normalized внутри в число конвертит и херит всю строку, при малейшем не соответствии формату, две точки к примеру 123..
There was a problem hiding this comment.
по хорошему, если юзер ставит точку где-то, надо точки остальные убирать и оставлять там, где курсор. Не хотел в эти дебри есть, поведение текущее меня устраивает.
There was a problem hiding this comment.
Так можно просто сравнить текущее значение с нормальным сохранённым и вернуть одно из них.
There was a problem hiding this comment.
было 123.321
ввели 123.3.21
сейчас вернется 1233.21
ты предлагаешь не давать юзеру простой способ переместить точку? надо обязательно сделать перед этим лишнее действие по удалению старой точки?
There was a problem hiding this comment.
И что значит сравнить с нормальным сохраненным? в next у нас 2 точки, как понять какую отбросить?
просто сравнение не поможет тут
Если сконвертить в число и проверить на NaN - то этого мало, т.к. эта проверка не сетит в value, но инпут не ограничивает, иначе какой-нить минус не ввести будет отдельно
There was a problem hiding this comment.
ты предлагаешь не давать юзеру простой способ переместить точку?
Странный вывод. Я предлагаю разрешить ему писать что хочет, но не проталкивать вглубь невалидные значения.
There was a problem hiding this comment.
я не понимаю, что ты хочешь, предлагаю закруглятся
number/number.view.ts
Outdated
|
|
||
| for (let i = num_str.length - 1; i >= 0; i--) { | ||
| result += '_' | ||
| if ((i % 3) === 0) result += ' ' |
There was a problem hiding this comment.
Я бы предложил форматировать через подчёркивания, как в JS.
There was a problem hiding this comment.
подчеркивания используются в mol_format как плейсхолдер в маске:
Line 28 in 96aeaf3
разделитель не может быть подчеркиванием
There was a problem hiding this comment.
Возможно стоит добавить кастомизацию плейсхолдера.
There was a problem hiding this comment.
такая себе идея, будут несовместимые шаблоны, я пока отвинтил format, бажный он, быстро пофиксить не получилось
No description provided.